Skip to content

cglm compatability#289

Open
jonathandw743 wants to merge 1 commit into
rougier:masterfrom
jonathandw743:cglm2
Open

cglm compatability#289
jonathandw743 wants to merge 1 commit into
rougier:masterfrom
jonathandw743:cglm2

Conversation

@jonathandw743

Copy link
Copy Markdown

No description provided.

@rougier

rougier commented Jun 20, 2025

Copy link
Copy Markdown
Owner

Not sure to see the point on the proposed changes.

@jonathandw743

jonathandw743 commented Jun 20, 2025

Copy link
Copy Markdown
Author

Not sure to see the point on the proposed changes.

To avoid naming collision with cglm.

@rougier

rougier commented Jun 20, 2025

Copy link
Copy Markdown
Owner

I woudl prefer to have the option to choose one name or the other.

@jonathandw743

Copy link
Copy Markdown
Author

I woudl prefer to have the option to choose one name or the other.

ok, that would involve having a
#define VEC_2 <ftgl_vec2 or vec2> depending on which option in chosen.
however, I would like to note that a lot of c libraries prefix public facing functions and types with the library name to avoid this issue, and don't support c++ namespaces to the same extent that this library does.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants